feat(trigger): data class and helper functions for lambda trigger events #159

michaelbrewer · 2020-09-07T08:25:47Z

Issue #, if available: #164

Description of changes:

Each Lambda trigger event there is a corresponding typed data class which doc strings extracted from the AWS documentation, some helper methods to updating values as well as decoding any embedded data. In the test suite there is alot a series of example trigger events.

Example screenshot of the docs:

CloudWatchLogsEvent example usage:

from aws_lambda_powertools.utilities.data_classes import CloudWatchLogsEvent

def handler(event: Dict[str, Any], context: LambdaContext):
    # CloudWatchLogsEvent provides a typed data class with a method to decode gzipped log messages
    logs_data = CloudWatchLogsEvent(event).parse_logs_data()
    # CloudWatchLogsDecodedData is another typed data class with docstrings and getters
    for log_event in logs_data.log_events:
        handle_log_message(log_event.message)

PreTokenGenerationTriggerEvent example usage:

def handler(event: Dict[str, Any], context: LambdaContext) -> Dict[str, Any]:
    # PreTokenGenerationTriggerEvent data class includes docstrings and helper methods
    pre_token_event = PreTokenGenerationTriggerEvent(event)

    claims_override_details = pre_token_event.response.claims_override_details
    if pre_token_event.user_name == "SpecialUser":
        # ClaimsOverrideDetails has convenient setters
        claims_override_details.claims_to_suppress = ["Email"]
        claims_override_details.claims_to_add_or_override = {"attribute": "value"}

    if "SpecialGroup" in pre_token_event.request.group_configuration.groups_to_override:
        # A setter method for only part of groupOverrideDetails
        claims_override_details.set_group_configuration_preferred_role("preferred_value")

    # Updates are made to the original dict
    return event

S3Event example usage:

from aws_lambda_powertools.utilities.data_classes import S3Event

def handler(event: Dict[str, Any], context: LambdaContext) -> Dict[str, Any]:
    # S3Event data class includes docstrings and helper methods
    s3_event = S3Event(event)

    # Convenience methods for popular attributes
    bucket_name = s3_event.bucket_name
    key = s3_event.object_key

    # Handle S3 event notification
    handle_record(bucket_name, key)

    return {"statusCode": 200, "body": f"Successfully processed {bucket_name}/{key}"}

APIGatewayProxyEvent example (V1 and V2 are supported):

from aws_lambda_powertools.utilities.data_classes import APIGatewayProxyEventV2

def handler(event: Dict[str, Any], context: LambdaContext) -> Dict[str, Any]:
    event = APIGatewayProxyEventV2(event)

    # Comprehensive property access with docs
    claim_value = event.request_context.authorizer.jwt_claim.get("claim_key")
    # Query helper with optional default_value if no parameter is found
    qs_value = event.get_query_string_value("param", default_value="Default value")
    # Header helper with optional default_value
    header_value = event.get_header_value("name")
    print(f"Handle event with claim_value: {claim_value}, qs_value: {qs_value}, header_value: {header_value}")

    return {"statusCode": 200, "body": "Success!"}

KinesisStreamEvent example with decoding data:

from aws_lambda_powertools.utilities.data_classes import KinesisStreamEvent

def handler(event: Dict[str, Any], context):
    kinesis_event = KinesisStreamEvent(event)
    for record in kinesis_event.records:
        # Kinesis data is base64 encoded so decode and load as json
        payload: dict = record.kinesis.data_as_json()
        print(f"Decoded payload: {payload}")

Checklist

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Put together an intial set of common lambda trigger events

codecov-commenter · 2020-09-07T08:53:38Z

Codecov Report

Merging #159 into develop will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop      #159     +/-   ##
===========================================
  Coverage   100.00%   100.00%             
===========================================
  Files           33        46     +13     
  Lines          918      2070   +1152     
  Branches        77        90     +13     
===========================================
+ Hits           918      2070   +1152

Impacted Files	Coverage Δ
...mbda_powertools/utilities/data_classes/__init__.py	`100.00% <100.00%> (ø)`
...bda_powertools/utilities/data_classes/alb_event.py	`100.00% <100.00%> (ø)`
.../utilities/data_classes/api_gateway_proxy_event.py	`100.00% <100.00%> (ø)`
...s/utilities/data_classes/cloud_watch_logs_event.py	`100.00% <100.00%> (ø)`
.../utilities/data_classes/cognito_user_pool_event.py	`100.00% <100.00%> (ø)`
...lambda_powertools/utilities/data_classes/common.py	`100.00% <100.00%> (ø)`
...s/utilities/data_classes/dynamo_db_stream_event.py	`100.00% <100.00%> (ø)`
...tools/utilities/data_classes/event_bridge_event.py	`100.00% <100.00%> (ø)`
...ols/utilities/data_classes/kinesis_stream_event.py	`100.00% <100.00%> (ø)`
...mbda_powertools/utilities/data_classes/s3_event.py	`100.00% <100.00%> (ø)`
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 063d8cd...e860731. Read the comment docs.

Tracked down some of the AWS docs to inline with the dict wrapper classes.

For cases when the dict name conflicts with a python builtin we should use a consistent method name

Add support for UserMigrationTriggerEvent and include better docstrings

Add support for CustomMessageTriggerEvent and PreAuthenticationTriggerEvent

Add support for PreTokenGenerationTriggerEvent and PostAuthenticationTriggerEvent

Changes: * Add some missing docs pull for various AWS Docs * Fix SQS mapping * Make docs consistent

jplock · 2020-09-10T01:28:47Z

API Gateway events?

michaelbrewer · 2020-09-10T01:58:27Z

API Gateway events?

Sure. I can add that.

michaelbrewer · 2020-09-10T06:34:57Z

API Gateway events?

@jplock - i added support for api gateway proxy events.

aws_lambda_powertools/utilities/trigger/cloud_watch_logs_event.py

For both SNS and S3 event notifications we actually include a single records, so we can have conveince metods to access popular resources

koiker · 2020-09-12T14:19:54Z

aws_lambda_powertools/utilities/trigger/cloud_watch_logs_event.py

+
+class CloudWatchLogsLogEvent:
+    def __init__(self, log_event: dict):
+        self._val = log_event


Before you assign the event, it's a good idea to check first if the event has the key/values that you expect and if the event version is the version that you can handle.
Current event formats: https://docs.aws.amazon.com/AmazonCloudWatch/latest/monitoring/cloudwatch-and-eventbridge.html

Sample event format:

{ "version": "0", "id": "c4c1c1c9-6542-e61b-6ef0-8c4d36933a92", "detail-type": "CloudWatch Alarm State Change", "source": "aws.cloudwatch", "account": "123456789012", "time": "2019-10-02T17:04:40Z", "region": "us-east-1", "resources": [ "arn:aws:cloudwatch:us-east-1:123456789012:alarm:ServerCpuTooHigh" ], "detail": { "alarmName": "ServerCpuTooHigh", "configuration": { "description": "Goes into alarm when server CPU utilization is too high!", "metrics": [ { "id": "30b6c6b2-a864-43a2-4877-c09a1afc3b87", "metricStat": { "metric": { "dimensions": { "InstanceId": "i-12345678901234567" }, "name": "CPUUtilization", "namespace": "AWS/EC2" }, "period": 300, "stat": "Average" }, "returnData": true } ] }, "previousState": { "reason": "Threshold Crossed: 1 out of the last 1 datapoints [0.0666851903306472 (01/10/19 13:46:00)] was not greater than the threshold (50.0) (minimum 1 datapoint for ALARM -> OK transition).", "reasonData": "{\"version\":\"1.0\",\"queryDate\":\"2019-10-01T13:56:40.985+0000\",\"startDate\":\"2019-10-01T13:46:00.000+0000\",\"statistic\":\"Average\",\"period\":300,\"recentDatapoints\":[0.0666851903306472],\"threshold\":50.0}", "timestamp": "2019-10-01T13:56:40.987+0000", "value": "OK" }, "state": { "reason": "Threshold Crossed: 1 out of the last 1 datapoints [99.50160229693434 (02/10/19 16:59:00)] was greater than the threshold (50.0) (minimum 1 datapoint for OK -> ALARM transition).", "reasonData": "{\"version\":\"1.0\",\"queryDate\":\"2019-10-02T17:04:40.985+0000\",\"startDate\":\"2019-10-02T16:59:00.000+0000\",\"statistic\":\"Average\",\"period\":300,\"recentDatapoints\":[99.50160229693434],\"threshold\":50.0}", "timestamp": "2019-10-02T17:04:40.989+0000", "value": "ALARM" } } }

Keys that can be checked before we initialize the object:

version

source

@koiker - I did not know cloud watch log events had any such variations

@koiker Those docs are for cloudwatch events sent to EventBridge. This class is handling cloudwatch log events and the encoded data.

aws_lambda_powertools/utilities/trigger/api_gateway_proxy_event.py

Use new DictWrapper abstract class for the data classes that wrap an event Dict

michaelbrewer · 2020-09-13T04:54:54Z

@heitorlessa - would it help to include helper methods which help with common tasks

eg: For SES events you might want to fetch the content from S3 by messageId and parse out the text/plain or text/html elements

import email
import quopri


# Configured S3 bucket name
s3_bucket = "..."
# Configured Object key prefix
key_prefix = "..."


def get_message_contents(message_id: str, content_type: str = "text/html") -> str:
    """ Load the email from S3 and then search through the
    message_contents for the by content type

    :param message_id: Is used for the key to look up the message on S3
    :param content_type: Content type of the payload to return defaults to `text/html`
    :return: Returns the email contents matching the specified content_type
    """
    try:
        obj = s3.Object(s3_bucket, key_prefix + message_id)
        message_contents = obj.get()["Body"].read().decode("utf-8")
    except ClientError as e:
        raise Exception("S3 ClientError:", e.response["Error"]["Message"])
    else:
        email_message = email.message_from_string(message_contents)
        if email_message.is_multipart():
            # Get matching content_type payload
            return next(
                (
                    quopri.decodestring(part.get_payload()).decode()
                    for part in email_message.walk()
                    if content_type in part.get_content_type()
                ),
                None,
            )
        else:
            # Fail over to payload
            return email_message.get_payload()

Add support for Kinesis stream events with a helper method to decode data

heitorlessa · 2020-09-13T17:54:02Z

@cakepietoast @nmoutschen

could you do a full review here sometime this week?

I'll have a quick look tomorrow but would appreciate some fresh eyes here while I can't dedicate some proper time to this yet.

PS: I'm raising a feature request for Lambda to publish a schema for all event sources so we could auto-generate most of it in the future (including for JS).

heitorlessa

Firstly, a BIG thank you @michaelbrewer -- It's a lot of work and thought that you put on here, we deeply appreciate it.

Small comment for now on data as json/decoded for CW Logs.

I need some time to think on the design - I don't want to rush, so give me a few days to pull this and play with it some more.

At a first glance, I'd like to see us having properties for the absolutely minimal necessary, where the additional docstring and getters pay off in terms of maintenance, user experience, and adding a new event source.

From the top of my head, I see a big value in having getters for

Nested keys - It's painful to remember more than one key only to fetch X data
Data transformation like you did on Kinesis b64, CloudWatch Logs b64+gzip+optionally json, Dynamo etc.
Ambiguous properties that would benefit from additional explanations since their names or values can be easily misinterpreted

heitorlessa · 2020-09-13T18:25:51Z

aws_lambda_powertools/utilities/trigger/cloud_watch_logs_event.py

+    def decode_cloud_watch_logs_data(self) -> CloudWatchLogsDecodedData:
+        """Decode, unzip and parse json data"""
+        payload = base64.b64decode(self.aws_logs_data)
+        decoded: dict = json.loads(zlib.decompress(payload, zlib.MAX_WBITS | 32).decode("UTF-8"))


Can we either make JSON deserialisation as another property, or optionally off?

Since this is a data class utility, it's possible they'll have non-JSON data too.

I'm fine having it on by default, though -- This will likely be true for Kinesis and API GW binary

@heitorlessa ok i split it up.

heitorlessa · 2020-09-13T19:02:00Z

Firstly, a BIG thank you @michaelbrewer -- It's a lot of work and thought that you put on here, we deeply appreciate it.

Small comment for now on data as json/decoded for CW Logs.

I need some time to think on the design - I don't want to rush, so give me a few days to pull this and play with it some more.

At a first glance, I'd like to see us having properties for the absolutely minimal necessary, where the additional docstring and getters pay off in terms of maintenance, user experience, and adding a new event source.

From the top of my head, I see a big value in having getters for

Nested keys - It's painful to remember more than one key only to fetch X data

Data transformation like you did on Kinesis b64, CloudWatch Logs b64+gzip+optionally json, Dynamo etc.

Ambiguous properties that would benefit from additional explanations since their names or values can be easily misinterpreted

One extra bit... if, however, the point is to expose all keys regardless for auto completion then it's simply a trade off we'll have to agree on as we add more event sources.

This would make sense if the end goal is to make this a separate package that other projects could benefit from.

heitorlessa · 2020-09-13T19:20:17Z

@heitorlessa - would it help to include helper methods which help with common tasks

eg: For SES events you might want to fetch the content from S3 by messageId and parse out the text/plain or text/html elements

import email

import quopri





# Configured S3 bucket name

s3_bucket = "..."

# Configured Object key prefix

key_prefix = "..."





def get_message_contents(message_id: str, content_type: str = "text/html") -> str:

    """ Load the email from S3 and then search through the

    message_contents for the by content type



    :param message_id: Is used for the key to look up the message on S3

    :param content_type: Content type of the payload to return defaults to `text/html`

    :return: Returns the email contents matching the specified content_type

    """

    try:

        obj = s3.Object(s3_bucket, key_prefix + message_id)

        message_contents = obj.get()["Body"].read().decode("utf-8")

    except ClientError as e:

        raise Exception("S3 ClientError:", e.response["Error"]["Message"])

    else:

        email_message = email.message_from_string(message_contents)

        if email_message.is_multipart():

            # Get matching content_type payload

            return next(

                (

                    quopri.decodestring(part.get_payload()).decode()

                    for part in email_message.walk()

                    if content_type in part.get_content_type()

                ),

                None,

            )

        else:

            # Fail over to payload

            return email_message.get_payload()

Not yet - I'd like to see more demand on these beforehand.

These (SES example) would also require IAM permissions, so there's more at play

michaelbrewer · 2020-09-13T22:36:00Z

@heitorlessa maybe once we have the schema it would replace the @Property access methods. With some very basic logic to generate python safe names from the schema. But I would like to keep that could in place with some warning that is might change later.

For getters and decoding it can just be for a small subset of frequently used elements. Like query string, headers, path and body decoding for proxies api calls.

michaelbrewer · 2020-09-13T22:36:47Z

@heitorlessa - would it help to include helper methods which help with common tasks

eg: For SES events you might want to fetch the content from S3 by messageId and parse out the text/plain or text/html elements

import email

import quopri





# Configured S3 bucket name

s3_bucket = "..."

# Configured Object key prefix

key_prefix = "..."





def get_message_contents(message_id: str, content_type: str = "text/html") -> str:

    """ Load the email from S3 and then search through the

    message_contents for the by content type



    :param message_id: Is used for the key to look up the message on S3

    :param content_type: Content type of the payload to return defaults to `text/html`

    :return: Returns the email contents matching the specified content_type

    """

    try:

        obj = s3.Object(s3_bucket, key_prefix + message_id)

        message_contents = obj.get()["Body"].read().decode("utf-8")

    except ClientError as e:

        raise Exception("S3 ClientError:", e.response["Error"]["Message"])

    else:

        email_message = email.message_from_string(message_contents)

        if email_message.is_multipart():

            # Get matching content_type payload

            return next(

                (

                    quopri.decodestring(part.get_payload()).decode()

                    for part in email_message.walk()

                    if content_type in part.get_content_type()

                ),

                None,

            )

        else:

            # Fail over to payload

            return email_message.get_payload()

Not yet - I'd like to see more demand on these beforehand.

These (SES example) would also require IAM permissions, so there's more at play

Maybe for this we just have a helper method when you can provide the contents of the s3 object.

Create BaseProxyEvent for some reusable code for the http proxy events Add support for ALB events

For handling CloudWatchLogsEvent log data split the Decode and decompress from the parse as CloudWatchLogsDecodedData

michaelbrewer · 2020-09-14T04:36:48Z

API Gateway events?

I also included ALB events and there is a BaseProxyEvent class that has the shared code between all three.

lgtm-com · 2020-09-14T06:13:57Z

This pull request introduces 1 alert when merging a746ddc into 063d8cd - view on LGTM.com

new alerts:

1 for __eq__ not overridden when adding attributes

heitorlessa · 2020-09-14T06:25:35Z

@heitorlessa - would it help to include helper methods which help with common tasks

eg: For SES events you might want to fetch the content from S3 by messageId and parse out the text/plain or text/html elements

import email

import quopri





# Configured S3 bucket name

s3_bucket = "..."

# Configured Object key prefix

key_prefix = "..."





def get_message_contents(message_id: str, content_type: str = "text/html") -> str:

    """ Load the email from S3 and then search through the

    message_contents for the by content type



    :param message_id: Is used for the key to look up the message on S3

    :param content_type: Content type of the payload to return defaults to `text/html`

    :return: Returns the email contents matching the specified content_type

    """

    try:

        obj = s3.Object(s3_bucket, key_prefix + message_id)

        message_contents = obj.get()["Body"].read().decode("utf-8")

    except ClientError as e:

        raise Exception("S3 ClientError:", e.response["Error"]["Message"])

    else:

        email_message = email.message_from_string(message_contents)

        if email_message.is_multipart():

            # Get matching content_type payload

            return next(

                (

                    quopri.decodestring(part.get_payload()).decode()

                    for part in email_message.walk()

                    if content_type in part.get_content_type()

                ),

                None,

            )

        else:

            # Fail over to payload

            return email_message.get_payload()

Not yet - I'd like to see more demand on these beforehand.

These (SES example) would also require IAM permissions, so there's more at play

Maybe for this we just have a helper method when you can provide the contents of the s3 object.

Sure, that would be fine.

lgtm-com · 2020-09-14T06:32:58Z

This pull request introduces 1 alert when merging 7b4ec44 into 063d8cd - view on LGTM.com

new alerts:

1 for __eq__ not overridden when adding attributes

We don't need to include `_decompressed_logs_data` and `_json_logs_data` in equality tests

to-mc

Lets change the "trigger" naming convention to something more descriptive. My suggestion is data_classes so instead of from aws_lambda_powertools.utilities.trigger import DynamoDBStreamEvent we can do from aws_lambda_powertools.utilities.data_classes import DynamoDBStreamEvent

aws_lambda_powertools/utilities/trigger/sns_event.py

aws_lambda_powertools/utilities/trigger/common.py

michaelbrewer · 2020-09-17T16:18:46Z

Lets change the "trigger" naming convention to something more descriptive. My suggestion is data_classes so instead of from aws_lambda_powertools.utilities.trigger import DynamoDBStreamEvent we can do from aws_lambda_powertools.utilities.data_classes import DynamoDBStreamEvent

@cakepietoast - I just choose this name as it was based on what I saw in other libraries. More accurately it is the data classes for lambda trigger events.

I will update this and make the other requested changes.

to-mc

I'm happy with this, just need to update the documentation before we release it. I'd be glad to contribute the docs piece, let me know @michaelbrewer.

heitorlessa · 2020-09-18T11:44:57Z

Let's create another PR for the docs - Implementation is solid, @cakepietoast is also happy with the direction.

Feel free to merge @cakepietoast when you can

As always, huge thanks @michaelbrewer for working on this.

Michael Brewer added 3 commits September 7, 2020 01:22

feat(trigger): class wrapper for events

a3e3019

Put together an intial set of common lambda trigger events

build: fix build for python 3.6

2c50acb

fix(linters): make the python linters happy

bdf4385

Michael Brewer added 10 commits September 7, 2020 08:36

tests: add missing test cases

112635d

docs: Include some docstrings

d9253f5

Tracked down some of the AWS docs to inline with the dict wrapper classes.

feat(trigger): initial cognito triggers

057c947

feat(trigger): add event_bridge_event

84b63a3

feat(trigger): use consistent getter name

c1d516b

For cases when the dict name conflicts with a python builtin we should use a consistent method name

refactor: less copies passed around

2aea86c

feat(trigger): add UserMigrationTriggerEvent

17906ab

Add support for UserMigrationTriggerEvent and include better docstrings

feat(trigger): cognito custom message and pre-auth

1c69762

Add support for CustomMessageTriggerEvent and PreAuthenticationTriggerEvent

feat(trigger): cognito pre token and post auth

b9cdd66

Add support for PreTokenGenerationTriggerEvent and PostAuthenticationTriggerEvent

tests(trigger): some extra checks

87277ba

michaelbrewer marked this pull request as ready for review September 9, 2020 10:42

chore(trigger): clean up and docs

92a3ca7

Changes: * Add some missing docs pull for various AWS Docs * Fix SQS mapping * Make docs consistent

Michael Brewer added 2 commits September 9, 2020 19:28

chore: consistent naming

f391461

feat(trigger): Add api gateway proxy events

b396e8c

michaelbrewer changed the title ~~feat(trigger): class wrapper for events~~ feat(trigger): data class and helper functions for lambda trigger events Sep 10, 2020

chore: Add more docs

553c48a

jplock reviewed Sep 12, 2020

View reviewed changes

aws_lambda_powertools/utilities/trigger/cloud_watch_logs_event.py Outdated Show resolved Hide resolved

Michael Brewer added 2 commits September 12, 2020 11:34

fix(trigger): better type hinting

7e6a3e5

feat(trigger): Add conveince methods

7a7e862

For both SNS and S3 event notifications we actually include a single records, so we can have conveince metods to access popular resources

koiker reviewed Sep 12, 2020

View reviewed changes

feat(trigger): Create DictWrapper

379b11f

Use new DictWrapper abstract class for the data classes that wrap an event Dict

feat(trigger): API gateway helper methods

cc180d3

feat(trigger): Kinesis stream event

3811356

Add support for Kinesis stream events with a helper method to decode data

heitorlessa reviewed Sep 13, 2020

View reviewed changes

Michael Brewer added 2 commits September 13, 2020 21:19

feat(trigger): Application load balancer event

3c41f36

Create BaseProxyEvent for some reusable code for the http proxy events Add support for ALB events

refactor(trigger): Split decompress and parse

35358ed

For handling CloudWatchLogsEvent log data split the Decode and decompress from the parse as CloudWatchLogsDecodedData

feat(trigger): Some attribte caching

a746ddc

fix(trigger): Init with __init__

7b4ec44

Michael Brewer added 2 commits September 13, 2020 23:52

fix(trigger): Add missing __eq__

6306436

We don't need to include `_decompressed_logs_data` and `_json_logs_data` in equality tests

feat(trigger): unquote_plus s3 object key

0ffb1be

to-mc reviewed Sep 17, 2020

View reviewed changes

aws_lambda_powertools/utilities/trigger/sns_event.py Outdated Show resolved Hide resolved

aws_lambda_powertools/utilities/trigger/common.py Outdated Show resolved Hide resolved

Michael Brewer added 2 commits September 17, 2020 13:42

refactor(data_classes): rename package from trigger

d0c9e5b

refactor(data_classes): Use DictWrapper consistently

e860731

michaelbrewer requested review from to-mc and heitorlessa September 17, 2020 21:05

to-mc added area/utilities feature New feature or functionality labels Sep 18, 2020

to-mc approved these changes Sep 18, 2020

View reviewed changes

heitorlessa approved these changes Sep 18, 2020

View reviewed changes

to-mc merged commit b654ca6 into aws-powertools:develop Sep 18, 2020

michaelbrewer deleted the feat-event-wrapper branch February 22, 2021 13:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(trigger): data class and helper functions for lambda trigger events #159

feat(trigger): data class and helper functions for lambda trigger events #159

michaelbrewer commented Sep 7, 2020 •

edited

Loading

codecov-commenter commented Sep 7, 2020 •

edited

Loading

jplock commented Sep 10, 2020

michaelbrewer commented Sep 10, 2020

michaelbrewer commented Sep 10, 2020

koiker Sep 12, 2020

michaelbrewer Sep 12, 2020

michaelbrewer Sep 12, 2020

michaelbrewer commented Sep 13, 2020 •

edited

Loading

heitorlessa commented Sep 13, 2020

heitorlessa left a comment

heitorlessa Sep 13, 2020

michaelbrewer Sep 14, 2020

heitorlessa commented Sep 13, 2020

heitorlessa commented Sep 13, 2020

michaelbrewer commented Sep 13, 2020

michaelbrewer commented Sep 13, 2020

michaelbrewer commented Sep 14, 2020

lgtm-com bot commented Sep 14, 2020

heitorlessa commented Sep 14, 2020

lgtm-com bot commented Sep 14, 2020

to-mc left a comment

michaelbrewer commented Sep 17, 2020

to-mc left a comment

heitorlessa commented Sep 18, 2020

feat(trigger): data class and helper functions for lambda trigger events #159

feat(trigger): data class and helper functions for lambda trigger events #159

Conversation

michaelbrewer commented Sep 7, 2020 • edited Loading

Description of changes:

codecov-commenter commented Sep 7, 2020 • edited Loading

Codecov Report

jplock commented Sep 10, 2020

michaelbrewer commented Sep 10, 2020

michaelbrewer commented Sep 10, 2020

koiker Sep 12, 2020

Choose a reason for hiding this comment

michaelbrewer Sep 12, 2020

Choose a reason for hiding this comment

michaelbrewer Sep 12, 2020

Choose a reason for hiding this comment

michaelbrewer commented Sep 13, 2020 • edited Loading

heitorlessa commented Sep 13, 2020

heitorlessa left a comment

Choose a reason for hiding this comment

heitorlessa Sep 13, 2020

Choose a reason for hiding this comment

michaelbrewer Sep 14, 2020

Choose a reason for hiding this comment

heitorlessa commented Sep 13, 2020

heitorlessa commented Sep 13, 2020

michaelbrewer commented Sep 13, 2020

michaelbrewer commented Sep 13, 2020

michaelbrewer commented Sep 14, 2020

lgtm-com bot commented Sep 14, 2020

heitorlessa commented Sep 14, 2020

lgtm-com bot commented Sep 14, 2020

to-mc left a comment

Choose a reason for hiding this comment

michaelbrewer commented Sep 17, 2020

to-mc left a comment

Choose a reason for hiding this comment

heitorlessa commented Sep 18, 2020

michaelbrewer commented Sep 7, 2020 •

edited

Loading

codecov-commenter commented Sep 7, 2020 •

edited

Loading

michaelbrewer commented Sep 13, 2020 •

edited

Loading